[Feat] 4.0 HomeView Design 반영 #480
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
wltnryu
requested review from
mooninbeom,
Dorii0513,
minjung0067,
JeEonPark and
Bluecho97
and removed request for
mooninbeom and
Dorii0513
wltnryu
added
PR-NeedReview
wltnryu
commented
Mar 18, 2025
|
|
||
| struct PaperInfo: Identifiable, Hashable { | ||
| struct PaperInfo: Identifiable, Hashable, Codable, Transferable { |
There was a problem hiding this comment.
Paper Cell을 Folder Cell로 드래그 앤 드롭하고, 해당 액션을 통해 문서의 위치를 이동하는 로직을 적용하기 위해 추가하였습니다!
wltnryu
commented
Mar 18, 2025
| @State private var isMainSelected: Bool = true | ||
| @State private var isFavoriteSelected: Bool = false | ||
| @State private var isTagSelected: Bool = false | ||
| @State private var selectedCategory: CategorySelection = .main |
There was a problem hiding this comment.
HomeListView에서 전체 / 즐겨찾기 / 태그 / 폴더 중 한 가지만 선택 가능해야 했기에, selectedCategory를 통해 케이스를 분리하였습니다!
wltnryu
commented
Mar 18, 2025
| if isMovingFolder { | ||
| let itemsToMove: [FileSystemItem] = selectedItems.isEmpty | ||
| if homeViewModel.isMovingFolder { | ||
| let itemsToMove: [PaperInfo] = homeViewModel.selectedItems.isEmpty |
There was a problem hiding this comment.
폴더 이동 변수를 Binding을 통해 넘기기 보다는 ViewModel이 해당 변수를 가지도록 하여 View 사이의 의존도를 낮추고자 했습니다! 근데 이건 한 번 크게 개편을 해야 할 것 같아요...... QA 할 때 기능을 해치지 않는 선에서 한 번 쓱 둘러보겠습니다 :)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
변경 사항
기존에 사용하던 검색 관련 코드를 정리하였습니다 ✨
무니가 새로운 검색 코드를 정리해 줘서, 기존에 사용하던 코드는 더이상 사용하지 않음을 확인하였습니다! 과감하게 삭제 갈겼으니 사라진 코드가 많지만 걱정 마세요 :)
HomeView의 레이아웃이 크게 개편되었습니다 ✨기존에 사용하던 InfoView는 모두 삭제하고, 상단 메뉴바를 제외한 HomeView를 크게 두 가지로 나누어 디자인과 같이 왼쪽을 폴더와 같은 문서 분류 기준, 오른쪽을 문서 리스트로 분리하였습니다! 그래서 기존에 사용했던
FileSystemItem구조체는 더이상 사용하지 않아요! (아직 파일은 삭제하지 않았습니다!)기존 로직을 전부 이동했어요 ✨
3.0 Reazy에서 사용하던 로직을 새로 개편된 레이아웃 Cell에 모두 적용했고, 이는 무니가 작성해 준 검색 뷰 로직을 참고하였습니다!
팀원에게 전달할 사항(Optional)
HomePDFCell디자인 편집을 부탁합니다!@mooninbeom 감사합니다......
close #477